Add more Prism related tests to cruby-bindings workflow#3609
Conversation
|
I'm not 100% sure if we want to do this. In some cases we have a circular dependency where we need to update Prism and CRuby at the same time. Do we merge to CRuby first, then send a PR to Prism? I'm not sure about the order of operations in that case. |
This should prevent Prism from breaking CRuby's CI.
14594b6 to
1d1ecbf
Compare
From my (limited) experience working on default gems, yes. There will be a PR including both library change AND CRuby test changes opened on I think the point of running prism compilation test on CI is to identify the need to do so before we merge a PR. |
|
@st0012 makes sense to me! |
|
This is fine, thanks @st0012 |
This should prevent Prism from breaking CRuby's CI.